Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: TS unit and integration tests #173

Merged
merged 9 commits into from
Jul 15, 2024

Conversation

alexpietsch
Copy link
Member

Resolves #49

In general, the tests should run. But they run very inconsistent.
E.g. the jasmine tests: Most of the time, I have to uninstall and reinstall the node_modules before every run. Otherwise, the etapi tests time out. (I expect the beforeEach and afterEach to run inconsistent).

Also, the results for almost all search tests are wrong (see #162). I wasn't able to figure out why.

And one last note: I skipped the import test, because the used file test-export.zip is missing.

So in general: We need to rework the tests, and find a more reliable way for the etapi tests for the starting and stopping of the server.

@alexpietsch alexpietsch self-assigned this Jun 9, 2024
@alexpietsch alexpietsch linked an issue Jun 9, 2024 that may be closed by this pull request
@alexpietsch alexpietsch requested a review from a team June 11, 2024 14:21
@meichthys meichthys requested a review from eliandoran July 8, 2024 04:57
@alexpietsch alexpietsch requested a review from a team July 11, 2024 19:31
@eliandoran eliandoran marked this pull request as draft July 15, 2024 18:18
@eliandoran eliandoran marked this pull request as ready for review July 15, 2024 18:21
@eliandoran
Copy link

@alexpietsch , thank you for your effort!

@eliandoran eliandoran merged commit eff6ca3 into develop Jul 15, 2024
1 check passed
@eliandoran eliandoran deleted the feat/ts-unit-and-integration-tests branch July 15, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bring back unit and integration tests
3 participants